textview: Stop playing shenanigans with the target list

This breaks GtkSourceView adding custom targets on top of GtkTextView,
but we'll fix that later.

selection: Make GtkTargetEntry not allocatable

And remove it being registered as a boxed type (lol).

HighContrast: misc fixes

The HighContrast theme was not parsing anymore, due to
leftover widget style properties, and some missed cleanups,
like -gtk-icon-effect. Also update for the new focus handling,
and make checks and radios sharp again.

Adwaita: Make checks and radios sharp again

The assets we have are 14x14, set -gtk-icon-size to reflect that.

Change icon-size properties

We no longer support registering custom icon sizes, so
we can make these properties just enums. This also lets
us specify them by nick in ui files. Nice!

image: Update docs

Mention the new icon-size style classes, update the docs
for removed arguments and return values, and add new
functions to the docs.

Document -gtk-icon-size

Add this property to the list of CSS properties.

Support -gtk-icon-size for builtins

This makes css icons more similar to themed icons, which
is the overall goal here.

Drop gtk_icon_size_lookup

Add a new, private gtk_image_get_image_size to replace it, and
update the remaining callers in a suitable way.

recentchooser: Use GIcon instead of GdkPixbuf

iconhelper: Query size via CSS

Instead of looking at the icon size, look at the CSS value for
-gtk-icon-size. Set style classes depending on icon size instead.

Trivially change Adwaita and HighContrast to report the same values as
before.

enums: Change GtkIconSize values

The new values are the ones we intend to keep. The old ones had
duplicated meanings and nobody knew which one to take.

image: Don't take icon size in set_from_definition()

filechooserwidget: icon size is always 16

filechooserbutton: icon size is always 16

So define it that way.

cellrendererspinner: Don't track old icon size

It was only used to avoid gtk_icon_size_lookup() calls and those aren't
expensive.

image: Remove icon-size argument from icon setters

Instead, add a function gtk_image_set_icon_size() for the cases where
overriding the icon size is necessary.

Treat icon sizes the same way as pixel sizes, too. So gtk_image_clear()
no longer unsets the icon size.

image: Change getters

Instead of returning the icon size with them, make
gtk_image_get_icon_name() and gtk_image_get_gicon() only return the icon
itself.

As a benefit, we can turn them into regular getters that return values
instead of requiring out parameters.

Instead, provide gtk_image_get_icon_size() to query the icon size.

button: Get rid of icon size

In fact, make gtk_button_new_from_icon_name() just set the icon-name
property instead of creating a GtkImage.

scalebutton: Remove icon size

toolbar: Remove icon sizes

tooltips: Remove GtkIconSize usage

css: Add -gtk-icon-size

This will replace GtkIconSize in future patches.

introspection: Fix build

wayland: Fix initial cursor

Make sure the initial cursor isn't random which would happen due to an
early exit when cursor == pointer->cursor triggered because both were
NULL.

gdk: Replace GDK_NONE with NULL

gdk: Turn GdkAtom into a const char *

Instead of an integer, it is now a char pointer. We also use
g_intern_string() instead of doing the interning ourselves.

gdk: Make GDK_NONE Atom translate to NULL

Instead of the string "NONE" we now use NULL.

gdk: Get rid of magic special atoms

x11: Remove special casing for GdkAtoms

This is in preparation for the GdkAtom removal

dnd: Make GtkDragDest and GtkDragSource use GtkTargetList

This gets rid of GtkTargetEntry in the API and consistently uses
GtkTargetList.

clipboard: Consistently use GtkTargetList

Some code was using GtkTargetList, some used GtkTargetEntry and some
GtkTargetPair.

selection: Remove #ifdef WAYLAND

Instead, turn the functions into backend API:
  gdk_broadway_display_add_selection_targets()
  gdk_broadway_display_clear_selection_targets()
Remove the old per-backend functions, too.

cellrendererpixbuf: Remove stock-detail property

GTK3 didn't use that property already...

messagedialog: Remove unused header

win32: Remove all checks for GDK_WINDOW_ROOT

We are no longer creating windows of this type, so no
need to check for it.

win32: Remove all uses of get_root_window

This is following similar changes done in the x11 and
broadway backends, but it has not been built.
Some fixups may be needed.

placessiderbar: Fix sunny compilation

Make compilation not warn about unused variables when compiling with
HAVE_CLOUDPROVIDERS undefined.

widget: No need to update events

GtkWindow queues for all events anyway, so the needed ones are already
there.

gdkwindow: Remove event_mask arguments from constructors

window: Set ALL_EVENT_MASK

This is a trivial commit that does a big change: We now ignore event masks.

Further commits will clean up code, but if bisection ends up here, you
know it's because code is getting delivered events that it weren't getting
before.

gdk: Fix GDK_ALL_EVENTS_MASK

This mask was forgotten to update when the last 2 event masks were
added, probably because it looks like it's already maxed.

widget: Remove gtk_widget_set_device_enabled()

Nobody is using it.

x11: Remove mentions of GDK_WINDOW_ROOT

This backend never creates a window of this type, so no
need to check for it anymore.

x11: Stop creating a root window

We no longer need it.

x11: Store toplevel list in GdkDisplay

This will let us get rid of the root window.

x11: Remove another use of the root window

We just need the X root window here.

icontheme: Add a trailing / when enumerating resources

This avoids extra string copies in GResource.

gtkplacessidebar: Fix new tab/window handling for cloud accounts

It wasn't taking into account whether the sidebar had support for them
or not, resulting in a file chooser with open in new tab/window menu
items when it's not supported.

To fix it, do as with the other menus and check for the availability of
new tab/window flags.

https://bugzilla.gnome.org/show_bug.cgi?id=786123

gtkplacessidebar: Adapt to libcloudproviders 0.2.x

And a few improvements on the way.

https://bugzilla.gnome.org/show_bug.cgi?id=786123

application: Append a / to the icon resource path

g_resources_enumerate_children expects the path to end
in a '/' (even though thats not stated in the docs), and
will copy it if that isn't the case. Avoid the copy
by putting  a '/' there to begin with.

icontheme: Append a / to the resource path

g_resources_enumerate_children expects the path to end
in a '/' (even though thats not stated in the docs), and
will copy it if that isn't the case. Avoid the copy
by putting  a '/' there to begin with.

a11y: Handle a treeview with no columns

Code was spewing criticals to stderr because of nonexisting columns. So
check that there's actually an existing column first.

treeview: Only update vadjustment in size_allocate()

The 2 calls even do different things, but because the one in allocate
always overrides the one here...

treeview: Pass height to allocate as argument

Instead of exporting a function to query it.

progresstracker: Don't hand out NaN

When the duration is set to 0, clamp it to 1us. This way we're almost
correct: We should really instantly finish, but we don't. But we do
respect the delay.

Doing this properly would require some refactoring of how the progress
tracker actually maintains progress, and this is just a quick fix.

treeview: Use gtk_widget_get_width()

... instead of gtk_widget_get_allocated_width() and same for height.

This makes the treeview do the right thing when used with padding.

treeview: Don't allocate columns outside of size_allocate()

treeview: Fix RTL column header allocation

treeview: width_changed is always true, so remove it

Also, sanitize the RTL correction code that made sure resizing the width
of a treeview would keep the contents glued to the right border instead
of the left border.

x11: Fix a crash

This was crashing the moment a second window is opened.

Fix up managed dnd

Since commit 6c56d04cee23d3340cf7cf08756c610efe9ccdab,
we were doing some calls twice in the managed dnd case,
leading to double drops.

textview: Remove GDK windows

textview: Simplify gtk_text_view_buffer_to_window_coords()

... and gtk_text_view_window_to_buffer_coords()

textview: Use text_window_get_width()/height()

Instead of poking the GdkWindow. They have the same size after all.

textview: Make the semi-private headrs really private

Rename the files to have the private.h ending.

And remove gtktextdisplay.h from the installed files.

textview: Store border window sizes in a custom struct

That way, we don't need to have a GtKRequisition in every TextWindow.

gtktexttag: Remove GtkTextTag::event and gtk_text_tag_event()

GtkSourceView is not using it, so it's unneeded. And it's certainly
diving deep into event internals of GtkTextView which hinders a proper
gesturization.

textview: Hide the selection bubble better

The selection bubble is not part of the text windows, so hiding it
during scroll should not be done in the text window code.

Also remove an unused variable that was only set in that code but never
read.

flowbox: Compute view box from correct variables

Drop the last use of GDK_MULTIHEAD_SAFE

This define is not used anymore, this was a leftover.

GtkCssImageIconTheme: Don't try to unref a NULL pointer

In gtk_css_image_icon_theme_snapshot(), don't try to unref icon_info
if it's NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=790171

rendericon: Add missing include

stylecontext: Make first property name explicit

Insist that a first non-NULL property is passed to
gtk_style_context_get().

This is in particular relevant because of dropping the state argument
since GTK3, and code like
  gtk_style_context_get (context, state, "font", &font);
would keep compiling without warnings without this change.

a11y: Report correct spinner size

calendar: Fix for GTK4 changes

testgtk: Draw the right area in alpha window test

Even broken stuff is easier now.

examples: Use right function to query size

testgtk: Fix some more old drawing area uses

tests: After 20 years, update the simple test to modern GTK

It's 3 lines shorter!
And it works again!

testgtk: Fix rotated text demo

Use proper DrawingArea APIs here.

x11: Reduce uses of the root window

Avoid calling the get_root_window api that returns
a GdkWindow in some places, and instead use the X
root window directly.

Simplify the gtk_snapshot_icon_texture API

Instead of passing the color matrix in from the outside,
just pass a boolean and set up the matrix internally.

Fix a compiler warning

Fix a typo

filesystem: Drop surface-based APIs

These are no longer used.

filechooserwidget: Use GIcons

This lets us avoid much of the symbolic icon rendering api.

filechooserbutton: Use GIcons

This lets use avoid much of the symbolic icon rendering api.

filesystem: Add GIcon getters

We want to move away from surfaces, and instead pass
GIcons and GdkTextures around.

Win32: Re-work cursor handling

Like the X11 and Wayland backends, re-work how the cursors are being
handled.  So, we use a hash table to cache up the HCURSORS that we
create along the way.

We still need to cache up the icon/cursor themes since this is something
that is not part of Windows but was added on to support icon/cursor themes
such as Adwaita on Windows, but should be in-line with what is going on in
GdkCursor.

Also, remove the _gdk_grab_cursor global variable in gdkprivate-win32.h,
and replace it with another variable in the GdkWin32Display structure,
to make things cleaner in the process.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Use color matrices for -gtk-recolor

Use the same approach we take for recoloring in GtkIconHelper now.
As part of this change, GtkCsSImageRecolor is changed to not derive
from GtkCssImageUrl anymore.

CssProvider: Fix doc of min theme version loaded

The code shows that we only load back to 4.0, but this still said 3.0.

CssProvider: Use consistent theme name placeholder

commit 475d916eb92c0106d09525bcca938f05fd6a81b5 added various paths that
use theme-name for this, but the existing path already used THEME, with
a subsequent description referring to the latter. So use that everywhere

Fix the build

css image: Use GdkTexture api directly

No need to load a pixbuf anymore, we can just
directly create textures.

css: Use a color matrix for recoloring -gtk-icontheme

This is the same approach we are already using in
GtkIconHelper.

Make the recoloring code more robust

Pass both width+height and scale, so we can handle the case
where width+height are zero and we are using the file, scaled.

fix the build

Fix symbolic.svg rendering

With the shader approach to symbolic recoloring, we must
not recolor the svgs anymore as we're loading them. Instead,
load them the same way that gtk-encode-symbolic-svg does.

This fixes the rendering of large symbolic icons e.g. the
'no search results found' page in the file chooser.

wayland: Drop an unused function

We no longer create a root window, so need to have
a function for it.